T_CONST ➔ ??? - Code Metrics - Inspection of "Browsers need it too ... :)" - geezify/geezify-js - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (b8a2ae)

by Samson

created 2019-04-24 06:41 UTC

T_CONST ➔ ??? A

↳ Parent: Geezify.js ➔ constructor

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	2
dl	0
loc	1
rs	10
c	0
b	0
f	0

const GeezConverter = require('./Converter/GeezConverter');

const AsciiConverter = require('./Converter/AsciiConverter');


/**
 * Geezify converts numbers in ASCII to Geez and vise versa.
 *
 * @author Sam As End <4sam21{at}gmail.com>
 */
module.exports = class Geezify {
    /**
     * Geezify constructor.
     *
     * @param $geez_converter
     * @param $ascii_converter
     */
    constructor($geez_converter, $ascii_converter) {
        this.geez_converter = $geez_converter;
        this.ascii_converter = $ascii_converter;
    }

    /**
     * Return a new Geezify instance.
     *
     * @return Geezify
     */
    static create() {
        return new Geezify(new GeezConverter(), new AsciiConverter());

    }

    /**
     * Converts ASCII number to geez.
     *
     * @param $ascii_number
     *
     * @throws \Geezify\Exception\NotAnIntegerArgumentException
     *
     * @return string
     */
    toGeez($ascii_number) {
        return this.geez_converter.convert($ascii_number);
    }

    /**
     * Convert geez to ASCII.
     *
     * @param string $geez_number
     *
     * @throws \Geezify\Exception\NotGeezArgumentException
     *
     * @return int
     */
    toAscii($geez_number) {
        return this.ascii_converter.convert($geez_number);
    }

    /**
     * @return GeezConverter
     */
    getGeezConverter() {
        return this.geez_converter;
    }

    /**
     * @param GeezConverter $geez_converter
     */
    setGeezConverter($geez_converter) {
        this.geez_converter = $geez_converter;
    }

    /**
     * @return AsciiConverter
     */
    getAsciiConverter() {
        return this.ascii_converter;
    }

    /**
     * @param AsciiConverter $ascii_converter
     */
    setAsciiConverter($ascii_converter) {
        this.ascii_converter = $ascii_converter;
    }
};


1			const GeezConverter = require('./Converter/GeezConverter');
			0 ignored issues – show Unused Code introduced 2018-05-15 16:05 UTC by Report Bug Copy Issue Report The constant `GeezConverter` seems to be never used. Consider removing it. Loading history...
2			const AsciiConverter = require('./Converter/AsciiConverter');
			0 ignored issues – show Unused Code introduced 2018-05-15 16:05 UTC by Report Bug Copy Issue Report The constant `AsciiConverter` seems to be never used. Consider removing it. Loading history...
3
4			/**
5			* Geezify converts numbers in ASCII to Geez and vise versa.
6			*
7			* @author Sam As End <4sam21{at}gmail.com>
8			*/
9			module.exports = class Geezify {
10			/**
11			* Geezify constructor.
12			*
13			* @param $geez_converter
14			* @param $ascii_converter
15			*/
16			constructor($geez_converter, $ascii_converter) {
17			this.geez_converter = $geez_converter;
18			this.ascii_converter = $ascii_converter;
19			}
20
21			/**
22			* Return a new Geezify instance.
23			*
24			* @return Geezify
25			*/
26			static create() {
27			return new Geezify(new GeezConverter(), new AsciiConverter());
			0 ignored issues – show Bug introduced 2018-05-15 16:05 UTC by Report Bug Copy Issue Report The variable `GeezConverter` seems to be never declared. If this is a global, consider adding a `/** global: GeezConverter /` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history... Bug introduced 2018-05-15 16:05 UTC by Report Bug Copy Issue Report The variable `AsciiConverter` seems to be never declared. If this is a global, consider adding a `/* global: AsciiConverter */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
28			}
29
30			/**
31			* Converts ASCII number to geez.
32			*
33			* @param $ascii_number
34			*
35			* @throws \Geezify\Exception\NotAnIntegerArgumentException
36			*
37			* @return string
38			*/
39			toGeez($ascii_number) {
40			return this.geez_converter.convert($ascii_number);
41			}
42
43			/**
44			* Convert geez to ASCII.
45			*
46			* @param string $geez_number
47			*
48			* @throws \Geezify\Exception\NotGeezArgumentException
49			*
50			* @return int
51			*/
52			toAscii($geez_number) {
53			return this.ascii_converter.convert($geez_number);
54			}
55
56			/**
57			* @return GeezConverter
58			*/
59			getGeezConverter() {
60			return this.geez_converter;
61			}
62
63			/**
64			* @param GeezConverter $geez_converter
65			*/
66			setGeezConverter($geez_converter) {
67			this.geez_converter = $geez_converter;
68			}
69
70			/**
71			* @return AsciiConverter
72			*/
73			getAsciiConverter() {
74			return this.ascii_converter;
75			}
76
77			/**
78			* @param AsciiConverter $ascii_converter
79			*/
80			setAsciiConverter($ascii_converter) {
81			this.ascii_converter = $ascii_converter;
82			}
83			};
84

geezify / geezify-js

Branch — master (b8a2ae)

T_CONST ➔ ??? A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like